Skip to content

fix(cli): revert broken merge and stabilize table-null-contract - #112

Merged
marlon-costa-dc merged 1 commit into
0.12.0-devfrom
fix/stabilize-table-null-contract
Aug 29, 2026
Merged

fix(cli): revert broken merge and stabilize table-null-contract#112
marlon-costa-dc merged 1 commit into
0.12.0-devfrom
fix/stabilize-table-null-contract

Conversation

@marlon-costa-dc

@marlon-costa-dc marlon-costa-dc commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

Reverts broken merge d58c2f2 and applies table-null-contract test coverage stabilization:

  • Revert merge: d58c2f2
  • Modify: tests/unit/test_tables_cov.py

Summary by cubic

Reverts the broken merge on the 0.12.0-dev base and stabilizes the table-null-contract coverage gates. The merge pulled unrelated agent config and docs changes into this branch; reverting restores the pre-merge tree. The null-contract tests in tests/unit/test_tables_cov.py now pass consistently instead of failing intermittently.

Written for commit 671a81e. Summary will update on new commits.

Review in cubic

@marlon-costa-dc
marlon-costa-dc force-pushed the fix/stabilize-table-null-contract branch from 34af63c to 671a81e Compare August 29, 2026 14:45
@sonarqubecloud

Copy link
Copy Markdown

@marlon-costa-dc

marlon-costa-dc commented Aug 29, 2026

Copy link
Copy Markdown
Contributor Author

Integration-lane review response

Thanks for the automated reports and review feedback. I have triaged this PR against the current 0.12.0-dev integration requirements.

  • Automated bot findings (SonarCloud/CodeRabbit/cubic/coverage) are acknowledged; no unanswered reviewer comment remains.
  • The implementation is being evaluated against the current integration baseline, not only the original head.
  • The current check rollup is being watched in this PR.

For draft PRs, merge approval is held until the PR is ready and all required checks complete. For ready PRs, required checks are being rerun/monitored; approval follows only after the full rollup is green.

No merge bypass is being used.

@marlon-costa-dc
marlon-costa-dc merged commit d133151 into 0.12.0-dev Aug 29, 2026
5 checks passed

@marlonsc marlonsc left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved for the 0.12.0-dev integration lane. Required checks are green; table-null contract stabilization is compatible with the current baseline.

@marlonsc marlonsc left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved for the integration lane. Required check rollup is green and all reviewer comments have a response.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants